…, cover reads

- const.py: add CONF_BIDIRECTIONAL ('bidirectional') and CONF_INITIAL_POSITION
- config_flow.py: async_step_blind now delegates to async_step_menu; add
  async_step_menu (shows 'Pair automatically' / 'Add manually' menu),
  async_step_manual_add (validates 2-hex enum, dedup check, mode, name),
  async_step_manual_position (initial position slider, timed only); extend
  __init__ with _pending_is_bidirectional/_pending_initial_position; add
  timed-motor guard at top of async_step_reconfigure (REVIEW-2)
- cover.py: add CONF_BIDIRECTIONAL/CONF_INITIAL_POSITION imports; read both
  in __init__ (CONF_BIDIRECTIONAL default True for legacy subentries, REVIEW-1);
  add extra_state_attributes returning mode; seed initial position in
  async_added_to_hass (RestoreEntity takes precedence)
- strings.json + translations/en.json: relabel initiate_flow.user to 'Add device'
  (REVIEW-3); add menu/manual_add/manual_position steps, invalid_enum_format +
  duplicate_enum errors, timed_calibration_unavailable abort
- tests/test_config_flow.py: fix handler binding (_make_handler helper with tuple
  handler + context source=user); all 11 Phase-2 tests now GREEN (34 pass)

…g, full gate

- tests/test_config_flow.py: add test_manual_add_enum_case_normalized (lowercase
  '1a' collides with existing '1A' after .upper()), test_manual_add_enum_stored_uppercase
  ('2b' stored as '2B')
- tests/test_cover.py: add test_cover_initial_position_clamped (CONF_INITIAL_POSITION=150
  clamps to 100; restored prior state of 50 beats seeded initial of 100)
- Full WSL pytest suite: 160 passed, 0 failures (no regressions)
- Native ruff: All checks passed; mypy: Success, no issues in 3 source files

… (CR-01)

In _async_position_update_loop, the position_reached branch now clears
_attr_is_opening, _attr_is_closing, _attr_is_closed, and _target_position
before returning -- matching the two boundary-exit branches that already
did this correctly. Without this fix, timed motors were left stuck in
a phantom opening/closing state indefinitely after SET_POSITION completed.

Also adds _unrecorded_attributes = frozenset({'mode'}) to SchellenbergCover
so the static mode attribute is excluded from the recorder (WR-05).

…ion, and cleanups (WR-01/03/04, IN-01/02)

- WR-01: async_step_manual_position now aborts with pairing_failed if
  _pending_device_enum is missing, preventing a broken empty-id subentry.
- WR-03: BooleanSelector default and resolver default for CONF_BIDIRECTIONAL
  flipped from False to True so an unmodified form yields bidirectional,
  matching the field-common case and the read-default used everywhere else.
- WR-04: removed the never-used _pending_initial_position attribute from __init__.
- IN-01: Awaitable moved from typing to collections.abc (deprecated in 3.9+).
- IN-02: Unicode en-dash in comment replaced with plain ASCII hyphen.

… orphaned menu key (WR-02, IN-04)

- WR-02: manual_position step description now notes that positioning for
  timed motors uses a default travel time and is uncalibrated until Phase 4.
- IN-04: removed the orphaned config.step.menu.menu_options.pair_device_menu
  key from both strings.json and translations/en.json; no matching flow step
  exists. Subentry-level user/manual_add keys are unaffected.

…and WR-03 default (WR-06)

- test_timed_motor_position_loop_clears_flags: exercises the REAL
  _async_position_update_loop for a timed motor with tiny travel time,
  drives SET_POSITION to 50, and asserts is_opening=False, is_closing=False,
  _target_position=None, and position==50 after completion (WR-06/CR-01).
- test_manual_position_aborts_without_pending_state: asserts abort with
  pairing_failed when manual_position is entered without prior manual_add (WR-01).
- test_manual_add_default_mode_is_bidirectional: asserts that omitting
  CONF_BIDIRECTIONAL from the form yields CONF_BIDIRECTIONAL=True (WR-03).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QwPMgtiypLgJ5nkR3mUGfL